Skip to content

chore: Add config with AGENTS.md context and opt-out label for greptile#2400

Merged
charlesvien merged 3 commits into
mainfrom
05-27-add_config_with_agents.md_context_and_opt-out_label
May 28, 2026
Merged

chore: Add config with AGENTS.md context and opt-out label for greptile#2400
charlesvien merged 3 commits into
mainfrom
05-27-add_config_with_agents.md_context_and_opt-out_label

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented May 27, 2026 •
edited
Loading

Problem

Greptile reviews this repo with no per-repo config, so it ignores our architecture rules. I'm matching this with the monorepo.

Changes

  1. Add greptile.json at repo root
  2. Feed AGENTS.md into every review via customContext.files
  3. Opt out per PR by applying the "No Greptile" label

How did you test this?

manually

Publish to changelog?

no

@charlesvien charlesvien changed the title Add config with AGENTS.md context and opt-out label chore: Add config with AGENTS.md context and opt-out label May 27, 2026
@charlesvien charlesvien changed the title chore: Add config with AGENTS.md context and opt-out label chore: Add config with AGENTS.md context and opt-out label for greptile May 27, 2026
@charlesvien charlesvien mentioned this pull request May 27, 2026
Merged
@charlesvien Graphite App
Copy link
Copy Markdown
Member Author

charlesvien commented May 27, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@charlesvien charlesvien mentioned this pull request May 27, 2026
Merged
@charlesvien charlesvien marked this pull request as ready for review May 27, 2026 22:34
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 27, 2026

T-Rex T-Rex Logs

What T-Rex did

  • Validated the new Greptile config shape against the documented config contract and confirmed that disabledLabels is supported and the No Greptile opt-out should work, and that customContext.files is documented as file objects with a path field rather than a bare string.
  • Read greptile.json and cross-checked it against official docs, noting the discrepancy where AGENTS.md would not be picked up because customContext.files is an array of objects with a path, and confirming case-insensitive label matching with globs for No Greptile.
Artifacts

Greptile config schema validation notes

  • Documents validation of the Greptile config against the contract, including disabledLabels handling and the required shape for customContext.files.

Greptile docs cross-check and URLs evidence

  • Summarizes the greptile.json read, official-docs comparison, and URLs checked to verify the exact corrected shape.

T-Rex Ran code and verified through T-Rex

Prompt To Fix All With AI 
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
greptile.json:4
**Use file objects** `customContext.files` is documented as an array of file objects with a `path` field, but this config passes `AGENTS.md` as a bare string. That can leave the file reference ignored or rejected, so reviews would not receive the architecture context this PR is trying to add.

```suggestion
        "files": [{ "path": "AGENTS.md" }]
```

Reviews (1): Last reviewed commit: "fix greptile.json indentation to 2 space..." | Re-trigger Greptile

greptile-apps[bot]
greptile-apps Bot reviewed May 27, 2026
View reviewed changes
Comment thread greptile.json Outdated
@charlesvien charlesvien force-pushed the 05-27-add_config_with_agents.md_context_and_opt-out_label branch from fcfb12c to 502a6b0 Compare May 28, 2026 00:36
@charlesvien charlesvien force-pushed the 05-27-update_usage_limit_modal_for_pro_users branch from 233cdcf to 3414316 Compare May 28, 2026 00:36
@charlesvien Graphite App
Copy link
Copy Markdown
Member Author

charlesvien commented May 28, 2026 •
edited
Loading

Merge activity

  • May 28, 1:10 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 28, 1:19 AM UTC: Graphite rebased this pull request as part of a merge.
  • May 28, 1:24 AM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien changed the base branch from 05-27-update_usage_limit_modal_for_pro_users to graphite-base/2400 May 28, 2026 01:11
@charlesvien charlesvien changed the base branch from graphite-base/2400 to main May 28, 2026 01:17
@charlesvien charlesvien force-pushed the 05-27-add_config_with_agents.md_context_and_opt-out_label branch from 502a6b0 to 3648667 Compare May 28, 2026 01:18
@charlesvien charlesvien merged commit 67edf96 into main May 28, 2026
18 checks passed
@charlesvien charlesvien deleted the 05-27-add_config_with_agents.md_context_and_opt-out_label branch May 28, 2026 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

@adboio adboio adboio approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charlesvien @adboio